Skip to content

Conversation

@zaneb
Copy link
Member

@zaneb zaneb commented Jul 6, 2022

With agent-based installation, we allow users to start directly from ZTP
CRs in cluster-manifests without first generating them from an
install-config. However, the latter will be an option, so resources will
need to depend on the InstallConfig without causing the survey to run if
it is not present.

@zaneb zaneb requested review from bfournie and pawanpinjarkar July 6, 2022 18:42
@openshift-ci openshift-ci bot requested review from celebdor and kirankt July 6, 2022 18:48
zaneb added 2 commits July 7, 2022 10:58
With agent-based installation, we allow users to start directly from ZTP
CRs in cluster-manifests without first generating them from an
install-config. However, the latter will be an option, so resources will
need to depend on the InstallConfig without causing the survey to run if
it is not present.
@zaneb zaneb force-pushed the optional-install-config branch 2 times, most recently from 43d410c to 5788406 Compare July 7, 2022 15:10
@zaneb
Copy link
Member Author

zaneb commented Jul 7, 2022

/retest-required

Copy link
Contributor

@pawanpinjarkar pawanpinjarkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 8, 2022
@lranjbar
Copy link
Contributor

lranjbar commented Jul 8, 2022

/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 8, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lranjbar

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 8, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 8, 2022

@zaneb: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-ci openshift-ci bot merged commit 30fb383 into openshift:agent-installer Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants